Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Adjust HTML page title to customized AppTitle #107

Merged
merged 2 commits into from
Aug 10, 2023

Conversation

sorcix
Copy link
Contributor

@sorcix sorcix commented Aug 10, 2023

The existing customize feature only changed the title in the navbar.

@sorcix
Copy link
Contributor Author

sorcix commented Aug 10, 2023

Oops, this causes an empty title tag when the customize option is not used. Fixing.

@sorcix sorcix marked this pull request as draft August 10, 2023 15:25
The existing customize feature only changed the title in the navbar.
@sorcix sorcix force-pushed the feature/customize-html-title branch from e776a01 to faccb17 Compare August 10, 2023 15:28
@sorcix sorcix marked this pull request as ready for review August 10, 2023 15:28
- Move application of fixes into loader function
- Remove default fallback from frontend source
- Fix documentation of default value in AppTitle

Signed-off-by: Knut Ahlers <knut@ahlers.me>
Copy link
Owner

@Luzifer Luzifer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! Thanks for bringing this up! 🙂

I added some small changes which doesn't alter the functionality of the fix, just some cleanups and a tiny change how the fix of an empty AppTitle is applied.

@Luzifer Luzifer changed the title Customize HTML page title Fix: Adjust HTML page title to customized AppTitle Aug 10, 2023
@Luzifer Luzifer merged commit 0ef9a57 into Luzifer:master Aug 10, 2023
3 checks passed
Luzifer added a commit that referenced this pull request Aug 10, 2023
Co-authored-by: Knut Ahlers <knut@ahlers.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants